Merge pull request #868 from GPSBabel/deldthrf

Delete destinator_itn destinator_poi destinator_trl easygps energympro enigma f90g flys ight fugawi from disuse.

Merge branch 'master' into deldthrf

Merge pull request #873 from tsteven4/clazy-use-static-qregularexpression

use static const qregularexpressions where possible

Merge pull request #869 from GPSBabel/delg

Delete geonet googledir gnav_trl ggv_ovl ggv_bin ggv_log glogbook gnav_trl hiketech

Merge pull request #865 from GPSBabel/nuke

Delete Delorme GPL format for disuse.

use cmake to run clazy.

this avoids warnings in Qt headers.

we also build both the CLI and GUI.

use static const QRegularExpressions in GUI.

use static const qregularexpressions where possible

update to Qt 6.2.4 (#872)

Update Russian translation (#864)

* Update Russian translation

* Update Russian translation

fix cross-origin resource sharing issues with map preview. (#871)

* fix cross-origin resource sharing issues with map preview.

This became critical in Qt 6.2.4 when using the file scheme
with gmapbase.html.

* Ensure QWebEngineSettings are set before page load.

Create SECURITY.md

Remove delgpl from CMakeFile

Delete tests from cmake. Fix tool for next time.

Delete formats from CMakeLists, too.

Remove destinator, engergypro, f90, crumbs from CMakefile.

Kill ggv_bin, ggv_ovl more dead.

Delete geo format.

Fix doc references left for EasyGPS.

Pick up dropped commit in nuke branch to drop gpl from vecs.

Remove entries from gpsbabel.qrc

Remove googledir, geonet

Take out ggv_bin, ggv_ovl, gnav_trl

Glogbook, Gnav_trl

Delete hiketech,

Delete destinator_itn destinator_poi destinator_trl easygps energympro enigma f90g flys ight fugawi from disuse.

Delete Delorme GPL for disuse.

Nits (#863)

* delete crumbs from deprecated formats.

* whitespace

retire strsub, gstrsub. (#862)

misc improvements. (#861)

* misc improvements.

fix memory leak in nmea reader.
eliminate extra string conversion using xml_parse_time.
use local QByteArray instead of xstrdup.
convert human_to_dec interface to QString.

* use strtod to track converted len in human_to_dec.

* enhance humantodec test.

Use PImpl idiom to hide Format and Filter headers from users of Vecs, FilterVecs. (#854)

* use pimpl to hide vec implementation of formats.

* use PIMPL for filter_vecs.

* sort vecs and filter_vecs include files.

* workaround memory leak issues with pimpl vecs, filter_vecs.

with some compilers sanitize address and valgrind reported errors.
The number of errors varied with compiler/platform.
Valgrind and sanitize address found different errors as well.
MacOS with clang 13 ran error free.
Linux with gcc9 and clang 10 showed multiple errors.
The errors detected were related to the destruction of the Format
and Filter instances.

* move ctors/dtors back to h files.

* catch nuke_format up with vecs.

convert gdb to Format class (#850)

* convert gdb to Format class.

* const member funcs

fix Qt6 deprecation warning with QVariant::canConvert. (#860)

silence MSVC with definitions and options (#859)

* deal with MSVC CRT security warnings in a consistent manner.

* correct cmake targets

* turn off some MSVC warnings for shapelib, zlib with cmake.

* fix msvc C4100 warning handling.

There was interference between our attempt to turn off the warning
with -wd4100 and win32-msvc mkspec changing the level of this
warning with -w34100.

fix various printf related issues. (#858)

* dont use posix printf argument reordering.

it isn't supported by MSVC printf, it requires _printf_p.

* fix lowranceusr printf errors on MSVC.

There are still multiple other errors with Qt6 on other platforms
related to the Qt5 -> Qt6 transition with sizes.

* fix size related printf issues in lowranceusr.

note the type of these variables is different between Qt5 and Qt6.

delete unused field from legacy format ff_vecs_t (#857)

use Qt for os independent environmental variable access. (#856)

Merge pull request #855 from GPSBabel/gpi_dbg

Move compile-time tests for debug message to runtime tests at -D3 or more.

Remove duplicate definition and undefinition.

Move compile-time tests for debug message to runtime tests at -D3 or more.

fix garmin_gpi debug code. (#853)

https://github.com/GPSBabel/gpsbabel/pull/833/files introduced a bug with unique_ptr.

use precompiled headers to speed up compile time (#851)

* speed up compiles with precompiled headers.

* workaround QTBUG with precompiled headers and clang.

* avoid including QtCore.

* add ability to turn off precompiled headers.

fix include cycle in new garmin_gpi Format. (#852)

eliminate unnecessary string conversions.

toString().to* is suspicious, QStringRef and QStringView probably
can be converted directly to *.

convert garmin gpi to Format class (#849)

* convert garmin_gpi to Format class

* const member funcs

* flush out includes

Use smart pointers over xmalloc and manual life management (#833)

* Use reference counted pointers instead of managing our own.
Fixes a leak when using debug printing.

update Qt 6 builds to 6.2.3 (#848)

convert text to Format class (#847)

convert humminbird to Format class. (#840)

* convert humminbird to Format class.

* const member functions, constexpr

* sorting members, easing off on forced init of auto variables.

* iwyu

convert f90g_track to Format class. (#844)

convert tef xml to Format class (#842)

* convert tef_xml to Format class.

* fix mem leak

* cap comment.

convert Mapbar track to Format class (#841)

* convert mapbar_track to Format class.

* const member funcs

convert mapfactor to Format class (#843)

* convert mapfactor to Format class.

* const memb funcs.

make kml skip and ignore list constexpr. (#846)

Convert gtrnctr to Format class (#839)

* Convert gtrnctr to Format class

* fix includes

* make ingorelist const

Convert gtrnctr to Format class (#839)

* Convert gtrnctr to Format class

* fix includes

* make ingorelist const

allow ignore list and skip list to be const. (#845)

convert exif format to Format class (#838)

* convert exif to Format class.

* exif member functions static, const.

* default initialize exif data members.

* move constant definition

convert html to Format class. (#837)

* convert html to Format class.

* add default init.

* capability comment

convert teletype to Format class. (#836)

* convert teletype to Format class.

* add default init.

Wintec tes format (#835)

* convert wintec_tes for Format class.

* add new h file.

* add default init.

move skytraq to Format class (#834)

* convert skytraq to Format class.

* make skytraq member functions const, use constexpr.

* skytraq format class fiddling

* more skytraq fiddling.

* whitespace cleanup

* rename skytraq base functions.

* fix rename, includes.

* capability comments.

Fix ODR violations discovered when using -flto. (#824)

The fedora build shows these:
skytraq.cc:666:8: warning: type 'struct read_state' violates the C++ One Definition Rule [-Wodr]
wbt-200.cc:127:8: note: a different type is defined in another translation unit
brauniger_iq.cc:30:6: warning: type 'state_t' violates the C++ One Definition Rule [-Wodr]
igc.cc:150:6: note: an enum with different value name is defined in another translation unit

However, even with link time optimization I was unable to observe a problem, i.e. testo passes.

There is a relevant note in https://en.cppreference.com/w/cpp/language/definition, see the note
about using unnamed namespaces to resolve this issue.

Also note the use of unnamed namespaces is not recommended in header files,
https://wiki.sei.cmu.edu/confluence/display/cplusplus/DCL59-CPP.+Do+not+define+an+unnamed+namespace+in+a+header+file

correct color palette issues in gpi writer. (#828)

simpify unix world conditionals in qmake files. (#831)

Replace which by POSIX command -v (#829)

a little lighter swing of the ax. (#826)

COPYING.txt is used by packaging.

fix cmake status messages (#825)

rc1 for 1.8.0 release (#814)

* rc1 for 1.8.0 release

* update gui about copyright.

* update translations again.

remove traces of autotools. (#822)

fix donate dialog. (#821)

enable hypertext link.
fix overelapping text on linux, windows.
fix "never show this message again".
fix donate hypertext link target.

finish killing GUI charset handling (#819)

* finish killing gui charset handling.

* more char set dead code cleaning

* even more dead code buried.

minor cleanups. (#820)

empty statements, unreachable code, member funct can be const.

capture git hash on github action builds. (#816)

* capture git hash on github action builds.

* modernize version sha display

* delete stray space

tweak cmake org target. (#815)

Move destinator to QDateTime from ctime (#796)

* Move destinator from ctime to QDateTime.

This feels a bit like rearranging Titanic chairs, but I needed to reduce
callers of strptime. This is another format I think we've rewritten a
couple of times without evidence of users. Most of the traffic in the last
ten years has been complaints of it not working

Destinator, consider yourself warned. You're on the death watch...

Remove trailing space in xmldoc/.
 sed --in-place 's/[[:space:]]\+$//' $*

 This is only the xmldoc part of 9935496f5c8c772ea11785ff489768dbb2de5d4f

Fix build on Catalina, HTML doc formatting improvements (#759)

* PDF maker: format screen and input same. Shrink padding slightly.

only support changing gpsbabel.org location at build generation time. (#813)

add dg100 heartbeat. (#812)

add support for cmake build options (#809)

* add initial support for cmake build options.

including basic library configuration.
Unlike our qmake build it generates and links with static libraries if
we are using our internally supplied copies.  The configuration options
are not as sophisticated as in our qmake build.  The configuration of
zlib will likely be acceptable as cmake provides support via findZLIB.
The configuration of shapelib and libusb-1.0 likely will likely need to
be enhanced.

* fix typo

* mimic qmake library searches.

too much energy went into verification with packagers that the
qmake options were sufficient for their needs.

* tweak library searches

add intial support for CTest (#808)

* add CTest tests.

* add note about improvements for ctest.

* running xmlwf and utf8bom tests is appropriate for single tests.

use hierarchy with cmake. (#807)

* use hierarchy with cmake.

* adjust cmake test script for hierarchy.

the default target with cmake builds both the CLI and the GUI.
alternatively, "cmake --build . --target gpsbabel" to build just the
CLI or "cmake --build . --target GPSBabelFE|gpsbabelfe to build just
the GUI.

* detect attempts to generate a build system from a subdirectory.

* fix shellcheck issues.

make gbversion.h, setup.iss generated files. (#806)

* make gbversion.h, setup.iss generated files.

add cmake documentation targets.

* work on check.

* simplify parameter passing to test scripts.

* cmake test work.

run vtesto in parallel.
fix valgrind decision making in testo.
fix test for gpsbabel executable with blanks in path.

* update vtesto for cmake.

* simplify option passing for testo.

* simplify valgrind detection in testo.

* fix testo bug.

* add ubuntu cmake build.

* fix workflow

* align gui pro and CMakeLists.

and use target properties

fix bad merge in mtk_logger. (#805)

1. Jan 01, 2017 269c55eb5 Robert fixes ftruncate issue.
2. Jan 23, 2017 604178aa8 HarelM fixes same a different way.
3. Jan 25, 2017 8456d1d21 Robert merges resulting in duplicate fixes.

I am backing out HarelM's fix.

Eliminate internal_styles.cc completely (#804)

* Eliminate internal_styles.cc entirely.

* add missing qrc file.

* fix resource compilation with ninja.

* fix potential detach.

* kill empty statement

* update qrc file formatting with QtCreator resource editor.

migrate from directory level cmake commands to target commands. (#802)

* migrate from directory level cmake commands to target commands.

fix a few bugs with our CMakeLists:

set WIN32 for gui target.  remove _CONSOLE, it's obsolete.

when undefining UNICODE, also undefine _UNICODE.  it's not clear to
me we have a sensitivity to UNICODE/_UNICODE.

take optimization flags out of CMakeLists, rely on CMAKE_BUILD_TYPE
to select appropriate flags.

don't force ZLIB_INHIBITED to be undefined on windows.

add cmake support for some of our configuration options:
disable-mappreview, embed_map, embed_translations.

* fix duplicated code in gui/CMakeLists.txt

* spelling fix for INSTALL.

Spelling fix in doc.

Merge branch 'master' of https://github.com/gpsbabel/gpsbabel

Spelling fix in interpolate: positve -> positive

take out some trash. (#799)

old configure scripts.
mingw stuff.

remove misleading, dangerous and unneeded redefinitions (#798)

for snprintf, vsnprintf.

delete an unneeded MSVC compiler pragma.

retire is_fatal for better static analysis (#797)

* join is_fatal lines in prep for transformation.

1,$g/is_fatal.*[^;]$/j

* replace is_fatal with if() fatal().

this allows better static analysis.

sed -i '1,$s/^\( *\)is_fatal *( *\(.*\), *MYNAME\([^;]*\);$/\1if (\2) {\n\1  fatal(MYNAME\3;\n\1}/' $file
sed -i '1,$s/^\( *\)is_fatal *( *\(.*\), *"\([^;]*\);$/\1if (\2) {\n\1  fatal("\3;\n\1}/' $file

* manual delete extra parenthesis.

Possible unterminated string in humminbird (#794)

Leave space for null terminator when copying string into fixed-size buffer.

Deprecate navicache (#793)

* Deprecate navicache.

use streams for garmin_fit debug messages. (#795)

* use streams for garmin_fit debug messages.

This eliminates the Qt 6 warnings related to the change in types
for container sizes and the format specifier.

* retire debug_print()

* make logging debug single arg ctors explicit

retire some csv char based routines. (#790)

* retire some csv char based routines.

The last csv_lineparse user, garmin_txt, is converted to csv_linesplit.

xcsv_parse_val converted to use csv_stringtrim(QString,QString,int),
eliminating the other overloads.

xcsv_parse_style_line parsing of FIELD_DELIMITER, FIELD_ENCLOSER,
RECORD_DELIMITER and BADCHARS updated to use the above overload of
csv_stringtrim.  FIELD_DELIMITER, FIELD_ENCLOSER and RECORD_DELIMITER
changed to trim, with a double quote enclosure,  before character substitution.
There was a subtle bug in csvs_stringtrim(char, char, int).  When trimming
a source that was all white space the first white space character would
be retained, while others would be trimmed.  This did occur with CRNEWLINE,
but the bug allowed other substitutions to work, e.g. SPACE, NEWLINE, TAB, CR.

A small bug in csv_stringclean was fixed.  If the to_nuke was empty an invalid
regular expression was created.

* save a few string conversions.

* and a few more conversions saved.

* review comments incorporated in garmin_txt reader.

* fix minor bug accumulating badchars.

fix xcsv parse_style formatting. (#789)

the blank line between the else and the if confused astyle.

support qt6 with cmake in windows build script. (#788)

organize source files in GPSBabel.pro, CMakeLists.text` (#787)

* organize files in GPSBabel.pro and CMakeLists.tst

This makes it easier to compare the two, and easier to see what
changed when nuking things.

* fix for lack of continuation line

* update nuke_format for sources on their own lines.

delete MSVC project and solution files.  The project files should
be generated with "qmake -tp vc GPSBabel.pro".  MSVC will create
the solution file if needed.